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commonly 
occurring 
solution to a 
recurring 
problem 




Pattern 



a solution to a 
problem that 
has negative 
consequences 



Anti-pattern 


easier to recognize what is 
wrong (and try to fix it), 
than to get it "right" in the 
first place 


Examples 

Spaghetti code: 

code with very complex, tangled control flow typified by 
lots of gotos 


Dead code: 

code whose “result" is no longer used 
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Refactoring 

Idea: 

change a software system so that the external behavior 
does not change but the internal structure is improved 


do this in small steps (change a bit and re-test) 


when adding a feature, refactor to make the addition easier 
to achieve 
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Code Smells 
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Bad Smells (in Code) 

Quote: 

“If it stinks, change it." 

— Grandma Beck on child rearing 










Exercise 


Question: 

What are some indicators or examples of poorly written 
code? 
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Bad Smells in Code 

Goal: 

critiquing code and software designs 


Suggested indicators: 

potential problems if left untouched 

solutions require judgment and balance 


Duplicated Code 

Indicator: 

the same functionality appearing in more than one place 

• e.g., same code in two methods of the same 
class 

• e.g., same code in two sibling subclasses 


Refactorings: 

Extract Method 

Pull Up Method 































Long Method 

Indicator: 

long, difficult-to-understand methods 


Why: 

desire “short", well-named methods 
cohesive units of code 

write a separate method instead of a comment 


Refactoring: 

Extract Method 


Large Class (Blob or God Class) 


Indicator: 

a class trying to do too many things 

• e.g., too many diverse instance variables 


Why: 

poor separation of concerns 


Refactoring: 

Extract Class 
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Divergent Change 

Indicator: 

when a class is commonly changed in different ways for 
different reasons 


Why: 

poor separation of concerns 


Refactoring: 

Extract Class 
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Shotgun Surgery 

Indicator: 

making a change requires many little changes across many 
different classes or methods 


Why: 

could miss a change 

should consolidate these changes 


Refactorings: 
Move Method 
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Long Parameter List 

Indicator: 

passing in lots of parameters to a method (because 
“globals are bad") 


Why: 

difficult to understand 


Refactorings: 

Replace Parameter with Method 
Introduce Parameter Object 
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Feature Envy 

Indicator: 

a method seems more interested in the details of a class 
other than the one it is in 

• e.g., invoking lots of get methods on another 
class 


int length = rect.getLength(); 
Why: int width = rect.getWidth(); 

this behavior may belon^W^the^her^S^^h * width; 


Refactorings. area = rect.area(); 

Move Method 

Extract Method 
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Data Class 


Indicator: 

classes that are just data (manipulated by other classes 
with getters and setters) 


Refactorings: 

Encapsulate Field 

Extract Method 

Move Method 
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Data Clumps 

Indicator: 

groups of data appearing together in the instance variables 
of classes, parameters to methods, etc. 


Refactorings: 

Extract Class 

public void doSomething( int x, int y, int z ) { 

Introduce Parameter Object 

} 
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Primitive Obsession 


Indicator: 

using the built-in types too much 

• e.g., “stringitus", everything being a string 


public static void checkCode( String postalCode ) { 

Why: 

leads to non-object-oriented designs 


Refactoring: 

Replace Data Value with Object 
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Switch Statements 


Indicator: 

long conditionals on type codes defined in other classes 


Refactorings: 

Extract Method, Move Method 
Replace Type Code 

Replace Conditional with Polymorphism 
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Speculative Generality 

Indicator: 

“we might need this someday" 

• e.g., an unused abstraction, hook, or parameter 


Why: 

increases design complexity 


Refactorings: 

Collapse Hierarchy 

Remove Parameter 
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Message Chains 

Indicator: 

long chains of navigation to get to an object 


.getB () .getC() .doSomething(); 

poor flexibility and testability 
could be Law of Demeter violation 


Refactoring: 

Hide Delegate 
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Inappropriate Intimacy 

Indicator: 

two classes that depend too much on each other, with lots of 
bidirectional communication 


Why: 

high coupling 

Refactorings: 
Move Method 

Extract Class 
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Refused Bequest 

Indicator: 

when a subclass inherits something that is not needed 

when a superclass does not contain truly common state or 
behavior 


Refactorings: 

Push Down Method and Push Down Field 
Replace Inheritance with Delegation 
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Comments 


Why: 

could be “deodorant" for bad smelling code 
simplify and refactor so comment is not needed 


use comments to explain why something was done a certain 
way 


Refactorings: 

Extract Method 

Rename Method 
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Refactoring Example 
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Refactoring Example 

Problem: 

a program to calculate and print a statement of a 
customer's charges at a video store: 

• customer can rent movies 

• movies have different pricing 

• movies are rented for a number of days 

• customer can collect frequent renter points 


what kind of design? 


Initial Structural Design 
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public class Movie { 

public static final int CHILDRENS = 2; 

public static final int REGULAR = 0; 

public static final int NEW_RELEASE = 1; 

private String _title; 

private int _priceCode; 

public Movie( String title, int priceCode ) { 

_t itle = title; 

_priceCode = priceCode; 

} 

public int getPriceCode() { 

return _priceCode; 

} 

public void setPriceCode( int arg ) { 

_priceCode = arg; 

} 

public String getTitleO { 
return _title; 

} 

} 
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public class Rental { 

private Movie _movie; 

private int _daysRented; 

public Rental( Movie movie, int daysRented ) { 

_movie = movie; 

_daysRented = daysRented; 

} 

public int getDaysRented() { 

return _daysRented; 

} 

public Movie getMovie () { 

return _movie; 

} 

} 
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public class Customer { 

private String _name; 

private Vector _rentals = new Vector(); 


public Customer( String name ) { 

_name = name; 

} 

public void addRental( Rental arg 
_rentals.addElement( arg ); 

} 

public String getName () { 

return _name; 

} 


{ 
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public String statement() { 

double totalAmount = 0; 
int frequentRenterPoints = 0; 

Enumeration rentals = _rentals.elements(); 

String result = "Rental Record for " + getName() + "\n"; 
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while (rentals.hasMoreElements0) { 

double thisAmount = 0; 

Rental each = (Rental)rentals.nextElement(); 

// determine amounts for each line 
switch (each.getMovie().getPriceCode()) { 

case Movie.REGULAR: 
thisAmount += 2; 
if (each.getDaysRented() > 2) 

thisAmount += (each.getDaysRented() - 2) * 1.5; 

break; 

case Movie.NEW_RELEASE: 
thisAmount += 

each.getDaysRented() * 3; 

break; 

case Movie.CHILDRENS: 
thisAmount += 1.5; 
if (each.getDaysRented() > 3) 

thisAmount += (each.getDaysRented() - 3) * 1.5; 

break; 

} 
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// add frequent renter points 
frequentRenterPointS++; 

// add bonus for new release rental 

if ((each.getMovie().getPriceCode() == Movie.NEW_RELEASE) && 

each.getDaysRented() > 1) 

frequentRenterPointS++; 

// show figures for this rental 

result += "\t" + each. getMovie () .getTitleO + "\t" + 

String.valueOf( thisAmount ) + "\n"; 
totalAmount += thisAmount; 
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// add footer lines 

result += "Amount owed is " + 

String.valueOf ( totalAmount ) + "\n"; 
result += "You earned " + 

String.valueOf( frequentRenterPoints ) + 
" frequent renter points"; 
return result; 

} 

} 
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Initial Behavioral Design 
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Code Smells 

What smells? 
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Code Smells 


Issues: 

procedural, not object-oriented programming 

statement () method does too much 
Customer class is a blob class 

potentially difficult to add features 




e.g., HTML output 
e.g., new charging rules 


Refactoring 

Idea: 

if the code is not structured conveniently to add a feature, 
first refactor the program to make it easy to add the 
feature, then add the feature 


small steps 
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Refactoring 

First step: 

need unit tests 
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Extract Method 


Goal: 

decompose statement () method 
extract logical chunk of code as a new method 


Extract Method (Before) 


public class Customer { 

public String statement() { 

double totalAmount = 0; 
int frequentRenterPoints = 0; 

Enumeration rentals = _rentals.elements(); 

String result = "Rental Record for " + getName() + "\n"; 

while (rentals.hasMoreElements 0) { 

double thisAmount = 0; 

Rental each = (Rental)rentals.nextElement() ; 

// determine amounts for each line 
switch (each.getMovie().getPriceCode()) { 

case Movie.REGULAR: 
thisAmount += 2; 
if (each.getDaysRented() > 2) 

thisAmount += (each.getDaysRented() - 2) * 1.5; 
break; 

case Movie.NEW_RELEASE: 

thisAmount += each.getDaysRented() * 3; 
break; 

case Movie.CHILDRENS: 
thisAmount += 1.5; 
if (each.getDaysRented() > 3) 

thisAmount += (each.getDaysRented() - 3) * 1.5; 
break; 

} 
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Extract Method (After) 

public class Customer { 

public String statement() { 

double totalAmount = 0; 
int frequentRenterPoints = 0; 

Enumeration rentals = _rentals.elements(); 

String result = "Rental Record for " + getName() + "\n"; 

while (rentals.hasMoreElements()) { 

double thisAmount = 0; 

Rental each = (Rental)rentals.nextElement(); 
thisAmount = amountEor( each ); 
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Extract Method (After) 

public class Customer { 

private double amountFor( Rental each ) { 

double thisAmount = 0; 

switch (each.getMovie().getPriceCode()) { 

case Movie.REGULAR: 
thisAmount += 2; 
if (each.getDaysRented() > 2) 

thisAmount += (each.getDaysRented() - 2) * 1.5; 

break; 

case Movie. NEW_RELEASE : 

thisAmount += each.getDaysRented() * 3; 

break; 

case Movie.CHILDRENS: 
thisAmount += 1.5; 
if (each.getDaysRented() > 3) 

thisAmount += (each.getDaysRented() - 3) * 1.5; 

break; 

} 

return thisAmount; 

} 


} 
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Extract Method 

Compile and test! 
small steps 
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Rename Variables 

Goal: 

rename variables in amountFor () 
enhance readability 


Rename Variables (Before) 

public class Customer { 

private double amountFor( Rental each ) { 

double thisAmount = 0; 

switch (each .getMovie ().getPriceCode()) { 

case Movie.REGULAR: 
thisAmount += 2; 
if (each .getDaysRented () > 2) 

thisAmount += (each .getDaysRented () - 2) * 1.5; 

break; 

case Movie. NEW_RELEASE : 

thisAmount += each.getDaysRented () * 3; 

break; 

case Movie.CHILDRENS: 
thisAmount += 1.5; 
if (each .getDaysRented () > 3) 

thisAmount += (each .getDaysRented () - 3) * 1.5; 

break; 

} 

return thisAmount; 

} 


} 
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Rename Variables (After) 

public class Customer { 

private double amountFor( Rental aRental ) { 

double result = 0; 

switch (aRental.getMovie (). getPriceCode0) { 

case Movie.REGULAR: 
result += 2; 

if (aRental.getDaysRented () > 2) 

result += (aRental.getDaysRented () - 2) * 1.5; 

break; 

case Movie.NEW_RELEASE: 

result += aRental.getDaysRented () * 3; 

break; 

case Movie.CHILDRENS: 
result += 1.5; 

if ( aRental.getDaysRented () > 3) 

result += (aRental.getDaysRented () - 3) * 1.5; 

break; 

} 

return result; 

} 


} 
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Rename Variables 



Compile and test. 
Anything unusual? 
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Move Method 


Refactoring: 

move amountFor () to Rental class 

• method uses rental information, but not 
customer information 

move this method to the right class 


Move Method (Before) 

public class Customer { 

private double amountFor( Rental aRental ) { 

double result = 0; 

switch (aRental .getMovie (). getPriceCode0) { 

case Movie.REGULAR: 
result += 2; 

if ( aRental.getDaysRented() > 2) 

result += ( aRental.getDaysRented() - 2) * 1.5; 

break; 

case Movie.NEW_RELEASE: 

result += aRental.getDaysRented() * 3; 

break; 

case Movie.CHILDRENS: 
result += 1.5; 

if ( aRental.getDaysRented() > 3) 

result += ( aRental.getDaysRented() - 3) * 1.5; 

break; 

} 

return result; 

} 


} 


51 



Move Method (After) 

public class Rental { 

public double getCharge() { 
double result = 0; 

switch (getMovieO . getPriceCode () ) { 

case Movie.REGULAR: 
result += 2; 

if (getDaysRented() > 2) 

result += (getDaysRented() - 2) * 1.5; 

break; 

case Movie.NEW_RELEASE: 

result += getDaysRented() * 3; 

break; 

case Movie.CHILDRENS: 
result += 1.5; 
if (getDaysRented() > 3) 

result += (getDaysRented() - 3) * 1.5; 

break; 

} 

return result; 

} 


} 
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Move Method (After) 

public class Customer { 

private double amountFor( Rental aRental ) { 

return aRental.getCharge(); 

} 


} 
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Move Method 



Compile and test. 
Cleanup indirection ... 
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Move Method (Continued) 

Refactoring: 

replace references to amountFor () with 
getCharge() 


adjust references to old method to use new method 
remove old method 
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Move Method (Continued) 

public class Customer { 

public String statement() { 

double totalAmount = 0; 
int frequentRenterPoints = 0; 

Enumeration rentals = _rentals.elements(); 

String result = “Rental Record for “ + getName() + “\n”; 

while (rentals.hasMoreElements()) { 

double thisAmount = 0; 

Rental each = (Rental)rentals.nextElement(); 
thisAmount = amountEor( each ) ; 
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Move Method (Continued) 

public class Customer { 

public String statement() { 

double totalAmount = 0; 
int frequentRenterPoints = 0; 

Enumeration rentals = _rentals.elements(); 

String result = “Rental Record for “ + getName() + “\n”; 

while (rentals.hasMoreElements()) { 

double thisAmount = 0; 

Rental each = (Rental)rentals.nextElement(); 
thisAmount = each.getCharge() ; 
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Move Method (Continued) 

Compile and test. 
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Replace Temp with Query 

Refactoring: 

eliminate thisAmount temporary in statement () 


replace redundant temporary variable with query 
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Replace Temp (Before) 

public class Customer { 

public String statement() { 

double totalAmount = 0; 
int frequentRenterPoints = 0; 

Enumeration rentals = _rentals.elements(); 

String result = "Rental Record for " + getNameO + "\n"; 

while (rentals.hasMoreElements0) { 

double thisAmount = 0; 

Rental each = (Rental)rentals.nextElement () ; 

thisAmount = each.getCharge(); 

// add frequent renter points 
frequentRenterPointS++; 

// add bonus for a two day new release rental 

if ((each.getMovie () .getPriceCode () == Movie.NEW_RELEASE) && 
each.getDaysRented() > 1) frequentRenterPoints++; 

// show figures for this rental 

result += "\t" + each. getMovie () .getTitleO + "\t" + 

String.valueOf( thisAmount ) + "\n"; 
totalAmount += thisAmount; 


Replace Temp (After) 

public class Customer { 

public String statement() { 

double totalAmount = 0; 
int frequentRenterPoints = 0; 

Enumeration rentals = _rentals.elements(); 

String result = "Rental Record for " + getNameO + "\n"; 

while (rentals.hasMoreElements0) { 

Rental each = (Rental)rentals.nextElement(); 

// add frequent renter points 
frequentRenterPointS++; 

// add bonus for a two day new release rental 

if ((each.getMovie().getPriceCode() == Movie.NEW_RELEASE) && 

each.getDaysRented() > 1) frequentRenterPoints++; 

// show figures for this rental 

result += "\t" + each.getMovie().getTitle() + "\t" + 

String.valueOf( each.getCharge() ) + "\n"; 

totalAmount += each.getCharge() ; 
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Extract/Move Method 

Refactoring: 

similarly, extract frequent renter points logic 




applicable rules go with the rental, not 
customer 


Extract/Move Method (Before) 


public class Customer { 

public String statement() { 

double totalAmount = 0; 
int frequentRenterPoints = 0; 

Enumeration rentals = _rentals.elements(); 

String result = "Rental Record for " + getNameO + "\n"; 

while (rentals.hasMoreElements0) { 

Rental each = (Rental)rentals.nextElement(); 

// add frequent renter points 
frequentRenterPointS++; 

// add bonus for a two day new release rental 

if ((each.getMovie().getPriceCode() == Movie.NEW_RELEASE) && 

each.getDaysRented() > 1) frequentRenterPoints++; 

// show figures for this rental 

result += "\t" + each.getMovie().getTitle() + "\t" + 

String.valueOf( each.getCharge() ) + "\n"; 

totalAmount += each.getCharge(); 
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Extract/Move Method (After) 


public class Customer { 

public String statement() { 

double totalAmount = 0; 
int frequentRenterPoints = 0; 

Enumeration rentals = _rentals.elements(); 

String result = "Rental Record for " + getName() + "\n"; 

while (rentals.hasMoreElements()) { 

Rental each = (Rental)rentals.nextElement(); 

// add frequent renter points 

frequentRenterPoints += each.getErequentRenterPoints() 
// show figures for this rental 

result += "\t" + each. getMovie () .getTitleO + "\t" + 
String.valueOf ( each.getCharge() ) + "\n"; 

totalAmount += each.getCharge(); 


Extract/Move Method (After) 


public class Rental { 

public int getFrequentRenterPoints() { 

if ((getMovie().getPriceCode() == Movie.NEW_RELEASE) && 

getDaysRented() > 1) 

return 2; 

else 

return 1; 

} 


} 
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Replace Temp with Query 

Refactoring: 

eliminate totalAmount temporary and replace with 

getTotalCharge () query 
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Replace Temp w/ Query (Before) 


public class Customer { 

public String statement() { 

double totalAmount = 0; 
int frequentRenterPoints = 0; 

Enumeration rentals = _rentals.elements(); 

String result = "Rental Record for " + getName() + "\n"; 

while (rentals.hasMoreElements 0) { 

Rental each = (Rental)rentals.nextElement(); 

// add frequent renter points 

frequentRenterPoints += each.getErequentRenterPoints(); 
// show figures for this rental 

result += "\t" + each.getMovie().getTitle() + "\t" + 
String.valueOf( each.getCharge() ) + "\n"; 

totalAmount += each.getCharge(); 

} 

// add footer lines 

result += "Amount owed is " + 

String.valueOf( totalAmount ) + "\n"; 
result += "You earned " + 

String.valueOf( frequentRenterPoints ) + 

" frequent renter points"; 
return result; 

} 

} 
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Replace Temp w/ Query (After) 


public class Customer { 

public String statement() { 

int frequentRenterPoints = 0; 

Enumeration rentals = _rentals.elements(); 

String result = "Rental Record for " + getName() + "\n"; 

while (rentals.hasMoreElements()) { 

Rental each = (Rental)rentals.nextElement() ; 

// add frequent renter points 

frequentRenterPoints += each.getFrequentRenterPoints(); 
// show figures for this rental 

result += "\t" + each. getMovie () .getTitleO + "\t" + 
String.valueOf( each.getCharge() ) + "\n"; 


// add footer lines 

result += "Amount owed is " + 

String.valueOf ( getTotalCharge() ) + "\n"; 

result += "You earned " + 

String.valueOf( frequentRenterPoints ) + 

" frequent renter points"; 
return result; 


Replace Temp w/ Query (After) 


public class Customer { 

private double getTotalCharge() { 

double result = 0; 

Enumeration rentals = _rentals.elements(); 

while (rentals.hasMoreElements()) { 

Rental each = (Rental)rentals.nextElement (); 

result += each.getCharge(); 

} 

return result; 

} 


} 
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Replace Temp with Query 

Refactoring: 

eliminate f requentRenterPoint s temporary and 
replace with 

getTotalFrequentRenterPoints () query 
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Replace Temp w/ Query (Before) 


public class Customer { 

public String statement() { 

int frequentRenterPoints = 0; 

Enumeration rentals = _rentals.elements(); 

String result = "Rental Record for " + getName() + "\n"; 

while (rentals.hasMoreElements()) { 

Rental each = (Rental)rentals.nextElement() ; 

// add frequent renter points 

frequentRenterPoints += each.getFrequentRenterPoints(); 
// show figures for this rental 

result += "\t" + each. getMovie () .getTitleO + "\t" + 
String.valueOf( each.getCharge() ) + "\n"; 


// add footer lines 

result += "Amount owed is " + 

String.valueOf ( getTotalCharge() ) + "\n"; 

result += "You earned " + 

String.valueOf( frequentRenterPoints ) + 

" frequent renter points"; 
return result; 
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Replace Temp w/ Query (After) 


public class Customer { 

public String statement() { 

Enumeration rentals = _rentals.elements(); 

String result = "Rental Record for " + getNameO + "\n"; 

while (rentals.hasMoreElements0) { 

Rental each = (Rental)rentals.nextElement() ; 

// show figures for this rental 

result += "\t" + each.getMovie().getTitleO + "\t" + 
String.valueOf( each.getCharge() ) + "\n"; 

} 

// add footer lines 

result += "Amount owed is"+ 

String.valueOf( getTotalCharge() ) + "\n"; 

result += "You earned " + 

String.valueOf( getTotalFrequentRenterPoints() ) + 

" frequent renter points"; 
return result; 

} 

} 


72 


Replace Temp w/ Query (After) 


public class Customer { 

private int getTotalFrequentRenterPoints() { 

int result = 0; 

Enumeration rentals = _rentals.elements(); 

while (rentals.hasMoreElements()) { 

Rental each = (Rental)rentals.nextElement(); 

result += each.getFrequentRenterPoints() ; 

} 

return result; 

} 


} 
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Initial Structural Design 
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After Extract Method 
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During Move Method 
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After Move Method 
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After Extract/Move Method 
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After Replace Temp w/ Query 
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After Replace Temp w/ Query 
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Second Structural Design 
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Second Behavioral Design 
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Refactoring 

Consequences: 

more object-oriented 

• decomposes big methods into smaller ones 

• distributes responsibilities among classes 


more code 

slower performance? 


New HTML Output Feature 

public class Customer { 

public String htmlStatement() { 

Enumeration rentals = _rentals.elements(); 

String result = "<hl>Rental Record for " + getNameO + "</hl>\n"; 

while (rentals.hasMoreElements0) { 

Rental each = (Rental)rentals.nextElement() ; 

// show figures for this rental 

result += each. getMovie () .getTitleO + ": " + 

String.valueOf( each.getCharge() ) + "<br>\n"; 

} 

// add footer lines 

result += "<p>Amount owed is " + 

String.valueOf( getTotalCharge() ) + "</p>\n"; 

result += "<p>You earned " + 

String.valueOf( getTotalErequentRenterPoints() ) + 

" frequent renter points</p>"; 
return result; 

} 

} 
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Changing Needs 

Feature: 

new price classifications of movies 


Move Method 


Refactoring: 

rental logic should not depend on specific movie types 
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Move Method (Before) 

public class Rental { 

public double getChargeO { 
double result = 0; 

switch (getMovie() .getPriceCode()) { 

case Movie.REGULAR: 
result += 2; 

if (getDaysRented() > 2) 

result += ( getDaysRented() - 2) * 1.5; 

break; 

case Movie.NEW_RELEASE: 

result += getDaysRented() * 3; 

break; 

case Movie.CHILDRENS: 
result += 1.5; 
if (getDaysRented() > 3) 

result += ( getDaysRented() - 3) * 1.5; 

break; 

} 

return result; 


} 
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Move Method (After) 

public class Movie { 

public double getCharge( int daysRented ) { 

double result = 0; 
switch (getPriceCode()) { 

case Movie.REGULAR: 
result += 2; 
if (daysRented > 2) 

result += (daysRented - 2) * 1.5; 

break; 

case Movie.NEW_RELEASE: 

result += daysRented * 3; 
break; 

case Movie.CHILDRENS: 
result += 1.5; 
if (daysRented > 3) 

result += (daysRented - 3) * 1.5; 

break; 

} 

return result; 


} 
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Move Method (After) 

public class Rental { 

public double getChargeO { 

return _movie.getCharge( _daysRented ); 

} 


} 
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Move Method (Before) 

public class Rental { 

public int getFrequentRenterPoints() { 

if ((getMovieO .getPriceCode() == Movie.NEW_RELEASE) && 

getDaysRented() > 1) 

return 2; 

else 

return 1; 

} 

} 
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Move Method (After) 

public class Movie { 

public int getFrequentRenterPoints( int daysRented ) { 

if ((getPriceCode() == Movie.NEW_RELEASE) && 

daysRented > 1) 
return 2; 

else 

return 1; 

} 

} 
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Move Method (After) 

public class Rental { 

public int getFrequentRenterPoints() { 

return _movie.getFrequentRenterPoints( _daYsRented ); 

} 


} 
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Replace Conditional Logic 

Ready for inheritance? ... 
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Proposed Redesign? 
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Proposed Redesign 

Flaw: 

a movie may change its classification during its lifetime 
(e.g., new release to regular) 

but, an object cannot change its class during its lifetime 


solution? 
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Replace Conditional Logic 

Idea: 

use Price (state) objects 
State design pattern 
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Replace Conditional Logic 
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Replace Type Code with State 

Refactoring: 

replace price (type) code 
compile and test after each step 

first, make sure uses of the price type code go through 
accessor methods ... 


Replace Type Code with State 


public class Movie { 

private int _priceCode; 

public Movie( String title, 
_title = title; 
_priceCode = priceCode; 

} 

public int getPriceCode() { 

return _priceCode; 

} 

public void setPriceCode( i 
_priceCode = arg; 

} 


} 


int priceCode ) { 


arg ) { 
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Replace Type Code with State 


public class Movie { 

private int _priceCode; 

public Movie( String title, 
_title = title; 
setPriceCode( priceCode ) 

} 

public int getPriceCode() { 

return _priceCode; 

} 

public void setPriceCode( i 
_priceCode = arg; 

} 


} 


Lnt priceCode ) { 


arg ) { 
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Replace Type Code with State 


Refactoring: 

add new state classes ... 
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Replace Type Code with State 


abstract class Price { 

public abstract int getPriceCode(); 

} 

class RegularPrice extends Price { 
public int getPriceCode() { 

return Movie.REGULAR; 

} 

} 

class NewReleasePrice extends Price { 
public int getPriceCode() { 

return Movie.NEW_RELEASE; 

} 

} 

class ChildrensPrice extends Price { 
public int getPriceCode() { 

return Movie.CHILDRENS; 

} 

} 


102 


Replace Type Code with State 


Refactoring: 

replace price type codes with instances of price state 
classes ... 


Replace Type Code with State 


public class Movie { 

private int _priceCode; 

public int getPriceCode() { 

return _priceCode; 

} 

public void setPriceCode( int arg ) { 

_priceCode = arg; 

} 


} 
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Replace Type Code with State 



public class Movie { 

private Price _price; 

public int getPriceCode() { 

return _price.getPriceCode() ; 

} 

public void setPriceCode( int arg ) { 

switch (arg) { 

case REGULAR: 

_price = new RegularPrice (); 
break; 

case NEW_RELEASE: 

_price = new NewReleasePrice(); 
break; 

case CHILDRENS: 

_price = new ChildrensPrice(); 
break; 
default: 

throw new IllegalArgumentException( 
"Incorrect price code" ) ; 

} 

} 
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Move Method 



Refactoring: 

move getCharge () to Price class 
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Move Method (Before) 

public class Movie { 

public double getCharge( int daysRented ) { 

double result = 0; 
switch (getPriceCode()) { 

case Movie.REGULAR: 
result += 2; 
if (daysRented > 2) 

result += (daysRented - 2) * 1.5; 

break; 

case Movie.NEW_RELEASE: 

result += daysRented * 3; 
break; 

case Movie.CHILDRENS: 
result += 1.5; 
if (daysRented > 3) 

result += (daysRented - 3) * 1.5; 

break; 

} 

return result; 


} 
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Move Method (After) 

public class Price { 

public double getCharge( int daysRented ) { 

double result = 0; 
switch (getPriceCode()) { 

case Movie.REGULAR: 
result += 2; 
if (daysRented > 2) 

result += (daysRented - 2) * 1.5; 

break; 

case Movie.NEW_RELEASE: 

result += daysRented * 3; 
break; 

case Movie.CHILDRENS: 
result += 1.5; 
if (daysRented > 3) 

result += (daysRented - 3) * 1.5; 

break; 

} 

return result; 


} 
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Move Method (After) 

public class Movie { 

public double getCharge( int daysRented ) { 

return _price.getCharge( daysRented ); 

} 


} 
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Replace Conditional with 
Polymorphism 

Refactoring: 

replace switch statement in getCharge () 


define abstract method 

for each case, add overriding method 



Replace Conditional with 
Polymorphism (Before) 

class Price { 

public double getCharge( int daysRented ) { 

double result = 0; 
switch (getPriceCode()) { 

case Movie.REGULAR: 
result += 2; 
if (daysRented > 2) 

result += (daysRented - 2) * 1.5; 

break; 

case Movie.NEW_RELEASE: 

result += daysRented * 3; 
break; 

case Movie.CHILDRENS: 
result += 1.5; 
if (daysRented > 3) 

result += (daysRented - 3) * 1.5; 

break; 

} 

return result; 

} 


} 
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Replace Conditional with 
Polymorphism (After) 

class RegularPrice { 

public double getCharge( int daysRented ) { 

double result = 2; 
if (daysRented > 2) 

result += (daysRented - 2) * 1.5; 

return result; 

} 

} 

class NewReleasePrice { 

public double getCharge( int daysRented ) { 

return daysRented * 3; 

} 

} 

class ChildrensPrice { 

public double getCharge( int daysRented ) { 

double result = 1.5; 
if (daysRented > 3) 

result += (daysRented - 3) * 1.5; 

return result; 

} 

} 
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Replace Conditional with 
Polymorphism (After) 

class Price { 

public abstract double getCharge( int daysRented ) ; 

} 
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Move Method 



Refactoring: 

move getFrequentRenterPoints () to Price 

class 
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Move Method (Before) 

public class Movie { 

public int getFrequentRenterPoints( int daysRented ) { 

if ((getPriceCode() == Movie.NEW_RELEASE) && 

daysRented > 1) 
return 2; 

else 

return 1; 

} 

} 


115 


Move Method (After) 

class Price { 

public int getFrequentRenterPoints( int daysRented ) { 

if ((getPriceCode() == Movie.NEW_RELEASE) && 

daysRented > 1) 
return 2; 

else 

return 1; 

} 

} 
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Move Method (After) 

public class Movie { 

public int getFrequentRenterPoints( int daysRented ) { 

return _price.getFrequentRenterPoints( daysRented ); 

} 


} 
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Replace Conditional with 
Polymorphism 

Refactoring: 

replace if statement in 

getFrequentRenterPoints() 
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Replace Conditional with 
Polymorphism (Before) 

class Price { 

public int getFrequentRenterPoints( int daysRented ) { 

if ((getPriceCode() == Movie.NEW_RELEASE) && 

daysRented > 1) 
return 2; 

else 

return 1; 


} 
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Replace Conditional with 
Polymorphism (After) 

class Price { 

public int getFrequentRenterPoints( int daysRented ) { 

return 1; 

} 


} 

class NewReleasePrice { 

public int getFrequentRenterPoints( int daysRented ) { 

return (daysRented >1) ? 2 : 1; 

} 


} 
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Refactoring 

Result: 

easier to change price behavior 

• change movie classifications 

• change rules for charging and frequent renter 
points 


rest of application does not know about this use of the State 
design pattern 
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Third Behavioral Design 


:Customer 


: Rental 


:Movie 


:Price 


Statement 


getTotalChargeO 


*[for all rentals] 
getCharge 
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getCharge( days) 


t} 


getCharge( days) 


getT otal F recjuentRenterPoi nts 


*[for all rentals] 
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Refactoring 

Principles 
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Refactoring 

Basic principles: 

catalog of refactorings 

do not change outward behavior 

reduce risk of change 
one thing at a time 
test each step 
iterate 


Refactoring 

Outcomes: 

encode design intent within class structure 


reorganizing code 
sharing logic 
express conditional logic 


Refactoring 

Potential limitations: 
too much indirection 

performance impact 

changing published interfaces 

are significant design changes possible? 


Refactoring 

When not to refactor: 

when you should rewrite 


when you are close to a deadline 


Refactoring 

An analogy: 

unfinished refactoring is like going into debt 


debt is fine as long as you can meet the interest payments 
(extra maintenance costs) 

if there is too much debt, you will be overwhelmed 


—Ward Cunningham 
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Redesigning with Patterns 

Common causes of change in client code: 

creating an object by naming the class directly 

• fix with Abstract Factory or Factory Method 
dependence on specific hard-code requests 

• fix with Chain of Responsibility or Command 
algorithmic dependencies 

• fix with Template Method 
tight coupling 

• fix with Fagade or Observer 
too much subclassing 

• fix with Decorator 
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Kinds of Refactorings 


Creating methods: 

intended to help reduce the size of methods and improve 
the readability of the code 


Extract Method, 

Inline Method, 

Replace Temp with Query 
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Kinds of Refactorings 


Moving features between objects: 

sometimes, responsibility is placed in the wrong class or a 
class ends up with too many responsibilities 


Move Method, 
Move Field, 
Extract Class 
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Kinds of Refactorings 


Organizing data: 

sometimes, objects can be used instead of simple data 
items 


Replace Data Value with Object, 
Replace Array with Object 
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Kinds of Refactorings 


Simplifying conditional expressions: 

conditional expressions and logic can be difficult to 
understand 


Replace Conditional with Polymorphism 


Kinds of Refactorings 


Making method calls simpler: 

complicated programming interfaces can be difficult to use 


Rename Method, 
Add Parameter 
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Kinds of Refactorings 


Dealing with generalization: 

getting methods and subclasses to the right place 


Pull Up Method, 
Push Down Method, 
Extract Subclass, 
Extract Superclass 
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Java Practices 

[Haggar, 2000]: 

general techniques 

objects and equality 

exception handling 

performance 

multithreading 

classes and interfaces 
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Java General Techniques 

Understand that all non-static methods can be overridden by 
default 

using final prevents a subclass from overriding a method 


Choose carefully between arrays and Vectors 

know their characteristics (element types, growable, 
speed) 
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Java General Techniques 

Prefer polymorphism to instanceof 

many uses of instanceof can be eliminated with 
polymorphism, which creates more extensible code. 


Use instanceof only when you must 
e.g., if you must safely downcast 


Java General Techniques 

Set object references to null when they are no longer needed 
even with garbage collection, still need to pay attention to 
memory usage 
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Java Classes and Interfaces 


Define and implement immutable classes judiciously 
sometimes want objects that do not change 

e.g., a color object 


how? 


Java Classes and Interfaces 


Enabling immutability for a class: 
declare all data private 

set all data in the constructor 

only getter methods; no setter methods 

declare the class final 


clone mutable objects before returning a reference to them 
from a getter method 

clone objects provided to the constructor that are 
references to mutable objects 
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Java Classes and Interfaces 



Use inheritance or delegation to define immutable classes from 
mutable ones 

reference a mutable object through an immutable interface 
• does not prevent casting the reference 


have an immutable object delegate to the mutable object 

have immutable abstract class and derived classes with 
mutable and immutable implementations 
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Effective Java 

[Bloch 2001]: 

creating and destroying objects 
methods common to all objects 
classes and interfaces 
substitutes for C constructs 
methods 

general programming 

exceptions 

threads 


serialization 


Effective Java 

Methods common to all objects: 

obey the general contract when overriding equals () 

always override hashcode o when you override equals 

always override tostring () 

override clone () judiciously 

consider implementing comparable 
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Effective Java 

Classes and interfaces: 

minimize the accessibility of classes and members 

favor composition over inheritance 

design and document for inheritance or else prohibit it 

prefer interfaces to abstract classes 

use interfaces only to define types 


Effective Java 

Methods: 

check parameters for validity 

make defensive copies when needed 

design method signatures carefully 

return zero-length arrays, not nulls 

write doc comments for all exposed API elements 


More Information 


Books: 

Refactoring 


• M. Fowler 

• Addison-Wesley, 1999 


AntiPatterns 

• W.J. Brown, R. C. Malveau, H. W. McCormick III, 
TJ. Mowbray 

• Wiley, 1998 
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More Information 


Books: 

Practical Java 

• P. Haggar 

• Addison-Wesley, 2000 

Effective Java 

• J. Bloch 

• Addison-Wesley, 2001 


More Information 


Articles: 

“Cloning Considered Harmful" Considered Harmful 

• C. Kapser and M. W. Godfrey 

• WCRE 2006 Proceedings, IEEE CS Press 


More Information 


Links: 

Refactoring Home Page 

• http://refactoring.com/ 


The 7 Deadly Sins of Software Development 

• http://www.javaworld.com/javaworld 
/jw-02-2011/110217-fatal-exception.html 


